Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature/356 checkov producer #369

Merged
merged 5 commits into from
Sep 24, 2024
Merged

Feature/356 checkov producer #369

merged 5 commits into from
Sep 24, 2024
+3,694 −28

Conversation

northdpole
Copy link
Contributor

@northdpole northdpole commented Sep 19, 2024 •
edited
Loading

close bug #367
close #356 create a producer for checkov

test with

make install
make publish-containers CONTAINER_REPO=localhost:5000/ocurity/dracon

in checkov's task.yaml replace the image with the published image
then run

bin/cmd/draconctl pipelines deploy ./examples/pipelines/checkov-project         
kubectl -n dracon create -f ./examples/pipelines/checkov-project/pipelinerun.yaml

@northdpole northdpole force-pushed the feature/356-checkov-producer branch 2 times, most recently from 1360e2d to 3716b08 Compare September 20, 2024 09:22
@northdpole northdpole marked this pull request as ready for review September 20, 2024 09:31
@northdpole northdpole linked an issue Sep 20, 2024 that may be closed by this pull request
cyclonedx package panics if the bom metadata does not contain the component purl #367
Closed
@northdpole northdpole force-pushed the feature/356-checkov-producer branch from 3716b08 to 1605c09 Compare September 23, 2024 08:36
@northdpole northdpole requested a review from ptzianos September 23, 2024 17:35
ptzianos
ptzianos reviewed Sep 23, 2024
View reviewed changes
components/producers/checkov/main.go Outdated Show resolved Hide resolved
components/producers/checkov/main.go Outdated Show resolved Hide resolved
components/producers/checkov/main.go Outdated Show resolved Hide resolved
components/producers/checkov/main.go Show resolved Hide resolved
components/producers/checkov/main_test.go Show resolved Hide resolved
components/producers/checkov/main_test.go Outdated Show resolved Hide resolved
pkg/cyclonedx/cyclonedx_test.go Outdated Show resolved Hide resolved
@northdpole northdpole force-pushed the feature/356-checkov-producer branch 2 times, most recently from e941923 to 1d4091c Compare September 23, 2024 18:35
@ptzianos ptzianos force-pushed the feature/356-checkov-producer branch 2 times, most recently from ceec694 to 822c7df Compare September 24, 2024 08:17
@ptzianos ptzianos force-pushed the feature/356-checkov-producer branch from 822c7df to 50472f9 Compare September 24, 2024 08:26
@northdpole northdpole merged commit d3568bd into main Sep 24, 2024
3 checks passed
@northdpole northdpole deleted the feature/356-checkov-producer branch September 24, 2024 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dlicheva dlicheva dlicheva approved these changes

@ptzianos ptzianos ptzianos approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@northdpole @ptzianos @dlicheva